Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Security for spoofed certs #98

Merged
merged 3 commits into from Nov 4, 2015
Merged

Security for spoofed certs #98

merged 3 commits into from Nov 4, 2015

Conversation

MikeBishop
Copy link
Contributor

Text giving recommendations for guarding against spoofed certs.

Require re-application of certificate pinning, mention possibility of
further implementation-specific checks.
@martinthomson
Copy link
Contributor

The text on pinning is fine. If the intent is expand on the authentication requirements for alternatives, the first sentence would suffice.

I'm concerned that the "additional requirements" you describe here might lead to interoperability problems. If these aren't specified, we are left to guess what the validation requirements are for any given client. I recognize that failure to authenticate an alternative doesn't necessarily translate into a fatal error - the alternative is just not used. However, I'm wondering what the motivation for the change is.

@reschke reschke added the alt-svc label Oct 7, 2015
@MikeBishop
Copy link
Contributor Author

This is from #76, which I should have tagged. You argued against the spec requiring an identical cert, and I agree that synchronization and security of the cert could certainly be an issue. However, the client might already have some data about the certs used by the origin and could reasonably expect some similarity between them.

As I said in the issue, I think cautious implementations will probably do something here, and I'd prefer the possibility to be mentioned in the spec so it's something operators will be aware of.

Text revision based on Yokohama discussion; replaced
"implementation-specific" with "other" and removed example.
@reschke
Copy link
Contributor

reschke commented Nov 4, 2015

@martinthomson are you ok with this proposal?

@martinthomson
Copy link
Contributor

Yes, it is what we discussed in the meeting.

reschke added a commit that referenced this pull request Nov 4, 2015
Security for spoofed certs (#98)
@reschke reschke merged commit 47c092d into httpwg:master Nov 4, 2015
reschke added a commit that referenced this pull request Nov 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

3 participants